Skip to content

Add yq to the kitchen sink and per-language Docker images - #756

Open
pierskarsenbarg wants to merge 4 commits into
mainfrom
add-yq
Open

Add yq to the kitchen sink and per-language Docker images#756
pierskarsenbarg wants to merge 4 commits into
mainfrom
add-yq

Conversation

@pierskarsenbarg

@pierskarsenbarg pierskarsenbarg commented Jul 13, 2026

Copy link
Copy Markdown
Member

We don't have an out-of-the-box way to merge multiple YAML files. Aaron Friel provided this suggestion in the past. This PR adds yq to provide that capability in the container images.

Adds yq to:

  • The pulumi/pulumi kitchen sink image
  • Go, Java, Node.js, Python, and .NET language runtime images

Note: The .ubi variants and base images were intentionally left untouched, as UBI9/microdnf repositories don't carry a yq package like Debian Trixie does.

@pierskarsenbarg
pierskarsenbarg requested a review from a team as a code owner July 13, 2026 14:05
@pierskarsenbarg pierskarsenbarg changed the title Added yq to the kitchen sink image Add yq to the kitchen sink and per-language Docker images Jul 13, 2026
@pierskarsenbarg

Copy link
Copy Markdown
Member Author

@iwahbe I've added yq to the language-based containers as well - is that ok or should it remain in the "kitchen sink" image only?

@tgummerer

Copy link
Copy Markdown
Contributor

Why do we need this in the per language containers? From the description this only really seems useful for yaml, so there shouldn't be any need to add it to language containers as well?

@pierskarsenbarg

Copy link
Copy Markdown
Member Author

Customer requesting this (and I guess others) are running YAML programs but have pre-run scripts that are in python and since the language containers are a little smaller, this suits their purpose.

@tgummerer

Copy link
Copy Markdown
Contributor

Personally I don't think this is right still. The language containers are intentionally small, with only the tools for the language installed. I don't think we should bloat them because some people use them for purposes they are not designed for, that's exactly what the kitchen sink container is supposed to be 🤔

@julienp

julienp commented Jul 14, 2026

Copy link
Copy Markdown
Contributor

Yeah, I agree with @tgummerer here. I can see it in kitchen sink and in the yaml image.

If we add yq everywhere, should we also add jq? What else? I prefer being strict here.

You can always extend an existing image.

@julienp

julienp commented Jul 14, 2026

Copy link
Copy Markdown
Contributor

Note: The .ubi variants and base images were intentionally left untouched, as UBI9/microdnf repositories don't carry a yq package like Debian Trixie does.

Ideally (not a blocker imo) we'd also add it to the UBI yaml image (I don't think there's an UBI kitchen sink).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants